Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add a second parameter to the largo_byline filter, containing an array of the arguments used in largo_byline() #1657

Merged
merged 2 commits into from
Mar 27, 2019
Merged

Add a second parameter to the largo_byline filter, containing an array of the arguments used in largo_byline() #1657

merged 2 commits into from
Mar 27, 2019

Conversation

benlk
Copy link
Collaborator

@benlk benlk commented Mar 27, 2019

Changes

  • The arguments set on largo_byline() are now passed to the largo_byline filter as an array of argument name => argument value. Pull request # for issue #1646.

@benlk
Copy link
Collaborator Author

benlk commented Mar 27, 2019

Tests are running slowly today.

@benlk benlk added this to the 0.6.2 milestone Mar 27, 2019
@benlk benlk merged commit 099fd2c into 0.5-dev Mar 27, 2019
@benlk benlk deleted the 1656-largo_byline-filter-args branch April 20, 2019 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
type: feature request type: improvement
Projects
None yet
Milestone
0.6.2
Development

Successfully merging this pull request may close these issues.
1 participant
@benlk